Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

manifest: update homekit revision #12660

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

manifest: update homekit revision #12660

wants to merge 1 commit into from
+1 −1

Conversation

mariuszpos
Copy link
Contributor

@mariuszpos mariuszpos commented Oct 12, 2023 •
edited
Loading

@github-actions github-actions bot added changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. manifest labels Oct 12, 2023
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 12, 2023 •
edited
Loading

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
homekit nrfconnect/sdk-homekit@v2.5.0-rc2 https://github.com/nrfconnect/sdk-homekit/pull/481 nrfconnect/sdk-homekit#481/files

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 12, 2023 •
edited
Loading

Test specification

CI/Jenkins/NRF

  • Integration Platforms

CI/Jenkins/integration

Test Module File based changes Manually selected West overwrite
test-sdk-homekit X

test-sdk-homekit: added because there was no .github/test-spec.yml in 'homekit'

Detailed information of selected test modules

Note: This message is automatically posted and updated by the CI

@NordicBuilder
Copy link
Contributor

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

Note: This comment is automatically posted by the Documentation Publishing GitHub Action.

@mariuszpos mariuszpos force-pushed the manifest_downgrade_prev_mcuboot branch from f47f205 to 68ce439 Compare October 12, 2023 07:45
@mariuszpos mariuszpos force-pushed the manifest_downgrade_prev_mcuboot branch from 68ce439 to c5aff29 Compare October 18, 2023 17:07
@mariuszpos mariuszpos force-pushed the manifest_downgrade_prev_mcuboot branch from c5aff29 to 5223da9 Compare October 19, 2023 06:58
@github-actions github-actions bot added doc-required PR must not be merged without tech writer approval. and removed changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Oct 19, 2023
@rakons rakons mentioned this pull request Oct 19, 2023
Merged
@mariuszpos mariuszpos removed the doc-required PR must not be merged without tech writer approval. label Oct 19, 2023
@mariuszpos mariuszpos added this to the 2.5.0 milestone Oct 19, 2023
@mariuszpos mariuszpos force-pushed the manifest_downgrade_prev_mcuboot branch from 5223da9 to ea16487 Compare October 19, 2023 10:34
@github-actions github-actions bot added the doc-required PR must not be merged without tech writer approval. label Oct 19, 2023
@mariuszpos mariuszpos force-pushed the manifest_downgrade_prev_mcuboot branch from ea16487 to 555905a Compare October 19, 2023 16:38
@github-actions github-actions bot added changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. and removed doc-required PR must not be merged without tech writer approval. labels Oct 19, 2023
@mariuszpos mariuszpos requested review from pjasinski990 and removed request for hakonfam, anangl and de-nordic October 19, 2023 16:38
manifest: update homekit revision
7250e51 
nrfconnect/sdk-homekit#481

Signed-off-by: Mariusz Poslinski <mariusz.poslinski@nordicsemi.no>
@mariuszpos mariuszpos force-pushed the manifest_downgrade_prev_mcuboot branch from 555905a to 7250e51 Compare October 24, 2023 14:45
@mariuszpos mariuszpos removed this from the 2.5.0 milestone Oct 24, 2023
@mariuszpos mariuszpos removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 24, 2023
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

Mariusz Poslinski seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sigvartmh sigvartmh sigvartmh approved these changes

@mariusz-nordicsemi mariusz-nordicsemi Awaiting requested review from mariusz-nordicsemi

@carlescufi carlescufi Awaiting requested review from carlescufi

@tejlmand tejlmand Awaiting requested review from tejlmand

@rlubos rlubos Awaiting requested review from rlubos

@nordicjm nordicjm Awaiting requested review from nordicjm

@pjasinski990 pjasinski990 Awaiting requested review from pjasinski990

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
DNM manifest manifest-homekit manifest-mcuboot
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@mariuszpos @NordicBuilder @CLAassistant @sigvartmh